Skip to content

keccak-hash: switch benches to criterion#315

Merged
ordian merged 8 commits into
masterfrom
ao-cargo-fix
Jan 14, 2020
Merged

keccak-hash: switch benches to criterion#315
ordian merged 8 commits into
masterfrom
ao-cargo-fix

Conversation

@ordian
Copy link
Copy Markdown
Contributor

@ordian ordian commented Jan 14, 2020

  • keccak-hash: switch benches to criterion
  • check benches on CI
  • apply cargo fix
  • updates docs to 2018 edition

Comment thread primitive-types/impls/serde/src/serialize.rs Outdated
Comment thread keccak-hash/benches/keccak_256.rs Outdated
Comment thread kvdb-rocksdb/Cargo.toml
[dev-dependencies]
alloc_counter = "0.0.4"
criterion = "0.3"
ethereum-types = { path = "../ethereum-types" }
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why was this needed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I broke this in https://github.com/paritytech/parity-common/pull/301/files#diff-3206254ae7bfdf4d978075eb4cafbdcfR30, but we didn't have CI checks to catch it because keccak-hash benches didn't compile on stable I think?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(this is a bench-only dependency)

Co-Authored-By: David <dvdplm@gmail.com>
@ordian ordian merged commit 687c04d into master Jan 14, 2020
@ordian ordian deleted the ao-cargo-fix branch January 14, 2020 23:25
ordian added a commit that referenced this pull request Jan 14, 2020
* master:
  keccak-hash: switch benches to criterion (#315)
dvdplm added a commit that referenced this pull request Feb 4, 2020
* master:
  update changelogs (#329)
  bump parity-util-mem to 0.4.2 (#328)
  remove libc feature from fixed-hash (#317)
  kvdb-rocksdb: release 0.4.2 (#327)
  kvdb-rocksdb: fix iter_from_prefix being slow (#326)
  MallocSizeOf for BTreeSet (#325)
  split off primitives (#323)
  travis: disable kvdb-web tests for chrome (#324)
  Expand const fn coverage (#319)
  uint: make zero const fn (#318)
  README: fix appveyor badge (#316)
  keccak-hash: switch benches to criterion (#315)
ordian added a commit that referenced this pull request Feb 6, 2020
* master: (56 commits)
  Remove libc completely (#333)
  update changelogs (#329)
  bump parity-util-mem to 0.4.2 (#328)
  remove libc feature from fixed-hash (#317)
  kvdb-rocksdb: release 0.4.2 (#327)
  kvdb-rocksdb: fix iter_from_prefix being slow (#326)
  MallocSizeOf for BTreeSet (#325)
  split off primitives (#323)
  travis: disable kvdb-web tests for chrome (#324)
  Expand const fn coverage (#319)
  uint: make zero const fn (#318)
  README: fix appveyor badge (#316)
  keccak-hash: switch benches to criterion (#315)
  update parity-util-mem (#309)
  Update features and feature dependencies (#307)
  Use proper memory queries to rocksdb (#308)
  Draft version updates and changelog (#299)
  Use custom error type for `from_hex` (#305)
  Fix typo. (#303)
  kvdb: remove KeyValueDBHandler (#304)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants